Skip to content

Add comments about redundant annotations - #1404

Merged
KFilipek merged 1 commit into
oneapi-src:mainfrom
ldorau:Add_missing_utils_annotate_memory_inaccessible
Jul 4, 2025
Merged

Add comments about redundant annotations#1404
KFilipek merged 1 commit into
oneapi-src:mainfrom
ldorau:Add_missing_utils_annotate_memory_inaccessible

Conversation

@ldorau

@ldorau ldorau commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Description

Add comments about redundant annotations.

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau changed the title Add missing utils annotate memory inaccessible Add missing annotations Jul 2, 2025
@ldorau

ldorau commented Jul 2, 2025

Copy link
Copy Markdown
Contributor Author

@ldorau ldorau mentioned this pull request Jul 2, 2025
11 tasks
@ldorau
ldorau force-pushed the Add_missing_utils_annotate_memory_inaccessible branch from d0417ec to b7c33a1 Compare July 2, 2025 09:00
@ldorau
ldorau marked this pull request as ready for review July 2, 2025 09:00
@ldorau
ldorau requested a review from a team as a code owner July 2, 2025 09:00
@ldorau
ldorau requested review from bratpiorka and lplewa July 2, 2025 09:01
@lplewa

lplewa commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Is it all missing annotations? We should mark memory as inaccessible after mmap, and only then mark it as undefined/defined, when given to the user.

@ldorau

ldorau commented Jul 3, 2025

Copy link
Copy Markdown
Contributor Author

Is it all missing annotations? We should mark memory as inaccessible after mmap, and only then mark it as undefined/defined, when given to the user.

@lplewa I think they are. Do you see any missing left?

@ldorau
ldorau force-pushed the Add_missing_utils_annotate_memory_inaccessible branch from b7c33a1 to 7bc845a Compare July 3, 2025 13:59
@ldorau ldorau closed this Jul 3, 2025
@ldorau ldorau reopened this Jul 4, 2025
Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau
ldorau force-pushed the Add_missing_utils_annotate_memory_inaccessible branch from 7bc845a to f1344d3 Compare July 4, 2025 06:01
@ldorau ldorau changed the title Add missing annotations Add comments about redundant annotations Jul 4, 2025
@ldorau
ldorau requested a review from KFilipek July 4, 2025 06:03
@KFilipek
KFilipek merged commit 70c33d0 into oneapi-src:main Jul 4, 2025
76 checks passed
@ldorau
ldorau deleted the Add_missing_utils_annotate_memory_inaccessible branch July 11, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants